[PR #1549/b53f7d72 backport][stable-6] s3_object - fix regression related to leading / in object key names #1554
Conversation
![patchback[bot]](https://avatars.githubusercontent.com/in/21488?s=60&v=4){kind=small}
…1549) s3_object - fix regression related to leading / in object key names SUMMARY fixes #1548 S3 object key names should not include a leading / (https://docs.aws.amazon.com/AmazonS3/latest/userguide/object-keys.html). However, for historical reasons we've supported them (we just pruned them out) this pruning was dropped in 6.0.0 without warning. For the sake of simplifying things and moving closer to the actual AWS resources, deprecate pruning out the leading /. (Arbitrarily modifying inputs and resource names tends to lead to strange edge cases) ISSUE TYPE Bugfix Pull Request COMPONENT NAME s3_object ADDITIONAL INFORMATION Regression of ansible/ansible#30576 / ansible/ansible#30579 Reviewed-by: Alina Buzachis (cherry picked from commit b53f7d7)
Docs Build 📝Thank you for contribution!✨ This PR has been merged and your docs changes will be incorporated when they are next published. |
|
Build succeeded. ✔️ ansible-galaxy-importer SUCCESS in 4m 16s |
|
Build succeeded (gate pipeline). ✔️ ansible-galaxy-importer SUCCESS in 3m 44s |
softwarefactory-project-zuul
bot
deleted the
patchback/backports/stable-6/b53f7d727509cc36de45d98ad0dd403a001d7a73/pr-1549
branch
… 'fake' module links (ansible-collections#1554)
This is a backport of PR #1549 as merged into main (b53f7d7).
SUMMARY
fixes #1548
S3 object key names should not include a leading
/(https://docs.aws.amazon.com/AmazonS3/latest/userguide/object-keys.html). However, for historical reasons we've supported them (we just pruned them out) this pruning was dropped in 6.0.0 without warning.For the sake of simplifying things and moving closer to the actual AWS resources, deprecate pruning out the leading
/. (Arbitrarily modifying inputs and resource names tends to lead to strange edge cases)ISSUE TYPE
COMPONENT NAME
s3_object
ADDITIONAL INFORMATION
Regression of ansible/ansible#30576 / ansible/ansible#30579